-
-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Try running tests as well #3
Conversation
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipe:
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
@conda-forge-admin, please rerender |
Hi! This is the friendly automated conda-forge-webservice. |
Tests pass on Linux/OSX but are failing on windows with what appears to be a fairly consistent message:
|
Related, perhaps: taleinat/fuzzysearch#13 |
Confusingly, I can't reproduce this with a local windows build of fuzzysearch 0.7.1. |
Hi @klauer, I just noticed this now... I'd be happy to help diagnose the failing tests, but unfortunately their outputs are no longer available on Azure Pipelines. I have a Windows machine available so I'll see if the tests happen to fail for me as well... |
@taleinat - no problem. Fixing the merge conflict here should trigger another build - let me know if you still can't see the output from there. For what it's worth, I had tried a while back to reproduce this locally on a Windows machine and couldn't. |
So, on the Windows build, for some reason at least one of the Cython modules appears very broken. I'm can't tell why, yet, as I too cannot reproduce this failure on my Windows machine. BTW, the Cython modules have their C code included in the source. Cython is not required at any stage. |
Interesting - I don't think I had noticed the inclusion of the cython-generated C source files. Is that common for cython projects? I might try a |
I've started working on this. The issue doesn't seem to be with the Cython-based extensions. |
@klauer, I've fixed what appears to be the source of this issue. The fix is available in version 0.7.3 which I've just released. Let's try this again with the new version. |
Great! Will bump the version and retry. |
Looks like it's still failing :( I see build warnings that I don't get when building locally on Windows. I'm using a slightly different version of MSCV: 14.11.25503 vs. version 14.16.27023 used in the Azure DevOps job, perhaps that's the reason? |
Hmm :/ it might be, but I don't believe we have control over that in the conda recipes. What's the differing warning you noticed? Happy to try any other fixes you can think of - patches can be relatively easily applied on top of v0.7.3 without touching the source repo. |
Is this being built with a standard version of Python or is it somehow customized? Is there a way to try this manually in a similar environment so that I can to diagnose this? |
In the feedstock, there's conda-forge releases its own Python builds (as does anaconda), but I don't believe they're modified from stock in any significant way. |
@conda-forge-admin, please rerender |
…da-forge-pinning 2020.10.13.19.09.25
Hi! This is the friendly automated conda-forge-linting service. I was trying to look for recipes to lint for you, but it appears we have a merge conflict. Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug. |
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
@conda-forge-admin, please rerender |
…nda-forge-pinning 2021.07.07.15.42.36
Hi! This is the friendly automated conda-forge-linting service. I was trying to look for recipes to lint for you, but it appears we have a merge conflict. Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug. |
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
@conda-forge-admin, please rerender |
…nda-forge-pinning 2023.11.07.18.09.01
Hi! This is the friendly automated conda-forge-linting service. I was trying to look for recipes to lint for you, but it appears we have a merge conflict. Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug. |
Checklist
conda-smithy
(Use the phrase@conda-forge-admin, please rerender
in a comment in this PR for automated rerendering)